Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nmc/1992 sharing #84

Closed
wants to merge 24 commits into from
Closed

Nmc/1992 sharing #84

wants to merge 24 commits into from

Conversation

TSI-amrutwaghmare
Copy link

@TSI-amrutwaghmare TSI-amrutwaghmare commented Jun 6, 2023

This PR Contains following modifications:

  • UI Customisation
  • Download limit functionality for link share
  • Unit test cases

Simulator Screenshot - iPhone 14 Pro Max - 2023-06-06 at 23 23 13
Simulator Screenshot - iPhone 14 Pro Max - 2023-06-06 at 23 23 18
Simulator Screenshot - iPhone 14 Pro Max - 2023-06-06 at 23 23 21
Simulator Screenshot - iPhone 14 Pro Max - 2023-06-06 at 23 23 28

@TSI-amrutwaghmare TSI-amrutwaghmare added custom MagentaCLOUD customisation approved labels Jun 6, 2023
@TSI-amrutwaghmare TSI-amrutwaghmare changed the title nmc 1992 - sharing Nmc/1992 - sharing Jun 12, 2023
@TSI-amrutwaghmare TSI-amrutwaghmare changed the title Nmc/1992 - sharing Nmc/1992 sharing Jun 12, 2023

var actions = [NCMenuAction]()

if share.shareType == 3, canReshare {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this spacing.

@objc func cancelClicked() {
delegate?.dismissShareAdvanceView(shouldSave: false)
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this spacing too.

labelQuickStatus.isHidden = true
imageDownArrow.isHidden = true
buttonMenu.isHidden = true
isUserInteractionEnabled = false

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to format the existing NC code as it will be considered as our customization.

isUserInteractionEnabled = false
switchCanEdit.isHidden = true
labelCanEdit.isHidden = true
buttonMenu.isHidden = true

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here no need to reformat this too.

@surinder-tsys
Copy link

Check if any code can be upstreamed to NC like DownloadLimit.

Same: check for other PRs where any code can be upstreamed to reduce customization.

@TSI-amrutwaghmare TSI-amrutwaghmare force-pushed the nmc/1992-sharing branch 2 times, most recently from 754ee45 to 86e0c3a Compare July 20, 2023 06:03
@TSI-amrutwaghmare TSI-amrutwaghmare added approved and removed approved custom MagentaCLOUD customisation labels Nov 21, 2023
@TSI-amrutwaghmare TSI-amrutwaghmare added the invalid This doesn't seem right label Jan 16, 2024
@TSI-amrutwaghmare TSI-amrutwaghmare deleted the nmc/1992-sharing branch February 13, 2024 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants